Skip to content

[FIX] Modify pipx inject command in README#93

Closed
rrebollo wants to merge 1 commit intoOCA:mainfrom
rrebollo:patch-1
Closed

[FIX] Modify pipx inject command in README#93
rrebollo wants to merge 1 commit intoOCA:mainfrom
rrebollo:patch-1

Conversation

@rrebollo
Copy link
Copy Markdown

Updated pipx inject command. Current instructions lead me to:

$ pipx inject --include-deps  oca-port git+https://github.com/OCA/odoo-module-migrator.git@master
Cannot pass --include-deps if --include-apps is not passed as well

Updated pipx command to include apps for oca-port.
Copy link
Copy Markdown

@edescalona edescalona left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sebalix
Copy link
Copy Markdown
Collaborator

sebalix commented Jan 9, 2026

Hello, thanks for this PR, but I cannot reproduce the issue locally, I tested with pipx version 1.4.3, and the latest 1.8.0.

My guess is you are using an older version of pipx, as the error message has been removed by this PR to make --include-apps implicit:

Can you try by upgrading your pipx installation?

@rrebollo
Copy link
Copy Markdown
Author

I think you are right. I don't remember it but it looks like I did upgrade pipx recently because I just test the installation instructions again and it's working as expected. So I'm closing this. Thank you.

@rrebollo rrebollo closed this Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants